Remove always-auth.#13377
Conversation
|
This line is still in the instructions for CFS. Are you sure it needs to be removed now? We could wait until we upgrade to a version of npm that fails. There is also a .npmrc in the |
We could wait 11 is the current version and it says it'll be removed in 12, but it said it was removed in npm 7 in 2021 because it wasn't used: https://docs.npmjs.com/cli/v7/using-npm/changelog#v7111-2021-04-23 |
…ft/vscode-cpptools into seanmcm/removeAlwaysAuth
There was a problem hiding this comment.
Pull request overview
Removes the deprecated always-auth=true setting from both .npmrc files to silence an npm warning.
Changes:
- Drop
always-auth=truefromExtension/.npmrc - Drop
always-auth=truefrom.github/actions/.npmrc
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| Extension/.npmrc | Removes deprecated always-auth option. |
| .github/actions/.npmrc | Removes deprecated always-auth option. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Fix npm warning: